Conversation
labkey-alan
left a comment
There was a problem hiding this comment.
Looks good to me, we may want to consider adding an integration test for this, and the release notes should be moved under the 3.0.0 section since this will be part of that release.
Adding an integration test would be great, but I don't know this package and the setup for that well enough to do it on my own. Could you help me out with that? I believe we'd need to 1) create a sample type or data class domain in container A 2) insert a row into that domain 3) create a child or sibling container that has the domain in scope 4) try the move_rows() |
Rationale
Related PR added a query-moveRows.api to the QueryController so that we could consolidate the various move entities actions to a single action. This PR adds move_rows() to the package.
Related Pull Requests
Changes